Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Use innerText instead of textContent #64

Merged
merged 1 commit into from
Feb 26, 2020

Conversation

iwanb
Copy link
Contributor

@iwanb iwanb commented Feb 26, 2020

To avoid copying styles and scripts, this makes it work with the sphinx-prompt extension.

https://www.w3schools.com/jsref/prop_node_innertext.asp

I hacked it like this in my conf.py file for now:

copybutton_prompt_text = """>>> ';
textContent = target.innerText.split('\\n'); var dummy = '
"""

To avoid copying styles and scripts, this makes it work with the sphinx-prompt extension.

https://www.w3schools.com/jsref/prop_node_innertext.asp

I hacked it like this in my conf.py file for now:
```
copybutton_prompt_text = """>>> ';
textContent = target.innerText.split('\\n'); var dummy = '
"""
```
@choldgraf
Copy link
Member

this seems very reasonable! Thanks for the PR :-)

@choldgraf choldgraf merged commit 1dfa97f into executablebooks:master Feb 26, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants